Skip to content

Fix lp mspm0g3519 pinctrl dtsi #98962

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
lsd-he360 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix lp mspm0g3519 pinctrl dtsi #98962

lsd-he360 wants to merge 2 commits into from
+2 −2

Conversation

@lsd-he360
Copy link

@lsd-he360 lsd-he360 commented Nov 6, 2025

The LP MSPM0G3519 board has additional pins that cannot be configured with entries defined in mspm0g1x0x_g3x0x-pinctrl.dtsi. The HAL was updated to include pinctrl definitions for TI gx51x family of MCUs.

@lsd-he360
west.yml: bump hal_ti to include gx51x pinctrl dtsi
717bafe 
Update hal_ti revision to include the gx51x pinctrl dtsi file. The file
has support for additional pins and peripherals not available in the
g1x0x_g3x0x family of MCUs.

Signed-off-by: Lay Desai <[email protected]>
@lsd-he360
boards: ti: lp_mspm0g3519: lp_mspm0g3519.dts: fix pinctrl dtsi
0d29646 
The mspm0g3519 has additional pins and peripherals that cannot be
configured with pinmux values defined in mspm0g1x0x_g3x0x-pinctrl.dtsi.
The HAL added pinctrl definitions for gx51x family of MCUs.

Signed-off-by: Lay Desai <[email protected]>
@github-actions
Copy link

github-actions bot commented Nov 6, 2025

Hello @lsd-he360, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@github-actions
Copy link

github-actions bot commented Nov 6, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_ti zephyrproject-rtos/hal_ti@cc04902 zephyrproject-rtos/hal_ti#76 zephyrproject-rtos/hal_ti#76/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_ti DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Nov 6, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 6, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@lsd-he360
Copy link
Author

lsd-he360 commented Nov 6, 2025

I tested the changes by running the echo bot sample.

echo-bot

@lsd-he360 lsd-he360 marked this pull request as ready for review November 6, 2025 20:49
@lsd-he360 lsd-he360 mentioned this pull request Nov 6, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ssekar15 ssekar15 Awaiting requested review from ssekar15

@vaishnavachath vaishnavachath Awaiting requested review from vaishnavachath

At least 2 approving reviews are required to merge this pull request.

Assignees

@vaishnavachath vaishnavachath

@ssekar15 ssekar15

Labels

area: Boards/SoCs DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_ti platform: TI Texas Instruments

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lsd-he360 @zephyrbot @vaishnavachath @ssekar15